Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Maintenance] Add CircleCI workflow to verify manifests - GT-493 #1431

Merged
merged 2 commits into from
Oct 11, 2023

Conversation

nikita-vanyasin
Copy link
Contributor

@nikita-vanyasin nikita-vanyasin commented Oct 10, 2023

  • CircleCI now uses Dynamic configuration to filter for modified files https://circleci.com/docs/using-dynamic-configuration/
  • When manifests or chart file changes, additional workflow will be execute which will spin up minikube and try to apply manifests
  • Fixed one issue found in customize for EE

@nikita-vanyasin nikita-vanyasin requested a review from a team as a code owner October 10, 2023 13:20
@cla-bot cla-bot bot added the cla-signed label Oct 10, 2023
@nikita-vanyasin nikita-vanyasin marked this pull request as draft October 10, 2023 13:20
@nikita-vanyasin nikita-vanyasin force-pushed the build/manifest-validation branch 3 times, most recently from 282823a to 726f52a Compare October 10, 2023 16:50
@nikita-vanyasin nikita-vanyasin force-pushed the build/manifest-validation branch from b18aef8 to 012fce2 Compare October 10, 2023 18:11
@nikita-vanyasin nikita-vanyasin changed the title Add CircleCI workflow to verify manifests [Maintenance] Add CircleCI workflow to verify manifests - GT-493 Oct 10, 2023
@nikita-vanyasin nikita-vanyasin marked this pull request as ready for review October 11, 2023 06:57
@jwierzbo jwierzbo merged commit ff34073 into master Oct 11, 2023
@jwierzbo jwierzbo deleted the build/manifest-validation branch October 11, 2023 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants